Skip to content

Enable collection filtering support for stac-auth-proxy #411

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 21, 2025

Conversation

GrzegorzPustulka
Copy link
Contributor

Added collection-search#filter conformance class to CollectionSearchExtension to enable compatibility with stac-auth-proxy collection filtering

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable
  • Changes are added to the changelog

Grzegorz Pustulka and others added 2 commits July 14, 2025 16:30
…xtension to enable compatibility with stac-auth-proxy collection filtering
Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. Just a couple thoughts.

BBrixen and others added 3 commits July 21, 2025 11:52
fixes incorrect or missing documentation for ES_USE_SSL,
ES_VERIFY_CERTS, and ES_API_KEY


**Description:**
- README.md documentation has incorrect default values for ES_USE_SSL
and ES_VERIFY_CERTS
- The default values that are used can be found at:
`./stac_fastapi/elasticsearch/stac_fastapi/elasticsearch/config.py` and
`./stac_fastapi/opensearch/stac_fastapi/opensearch/config.py`
- Adds documentation for ES_API_KEY

**PR Checklist:**

- [X] Code is formatted and linted (run `pre-commit run --all-files`)
- [X] Tests pass (run `make test`)
- [X] Documentation has been updated to reflect changes, if applicable
- [X] Changes are added to the changelog

---------

Co-authored-by: Bennett Patrick Brixen <bpbrixe@sandia.gov>
Co-authored-by: Jonathan Healy <jonathan.d.healy@gmail.com>
@jonhealy1 jonhealy1 self-requested a review July 21, 2025 11:00
Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jonhealy1 jonhealy1 merged commit e34e923 into stac-utils:main Jul 21, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants